Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Internal.AspNetCore.Sdk as an MSBuild SDK #905

Merged
merged 2 commits into from
Oct 4, 2018
Merged

Conversation

natemcmaster
Copy link
Contributor

@natemcmaster natemcmaster commented Oct 3, 2018

Convert from using Internal.AspNetCore.Sdk as a package reference to using it as an MSBuild SDK. This gives Internal.AspNetCore.Sdk more control over the projects and how they operation. MSBuild SDKs evaluate before restore, and can alter MSBuild settings which packages cannot.

cref aspnet/BuildTools#729

Nate McMaster added 2 commits October 3, 2018 11:13
…using it as an MSBuild SDK

This gives Internal.AspNetCore.Sdk more control over the projects and how they operation. MSBuild SDKs evaluate before restore, and can alter MSBuild settings which packages cannot.
dougbu added a commit to aspnet/Mvc that referenced this pull request Oct 4, 2018
- should resolve issues with occasional strange MSBuild caching issues in this repo
- modeled after dotnet/Scaffolding#905
- follows aspnet/BuildTools#729 recommendation to check in global.config file
  - see also dotnet/msbuild#2914
@natemcmaster natemcmaster merged commit ba6e9f5 into release/2.2 Oct 4, 2018
@natemcmaster natemcmaster deleted the namc/sdk branch October 4, 2018 16:22
dougbu added a commit to aspnet/Mvc that referenced this pull request Oct 4, 2018
- should resolve issues with occasional strange MSBuild caching issues in this repo
- modeled after dotnet/Scaffolding#905
- follows aspnet/BuildTools#729 recommendation to check in global.config file
  - see also dotnet/msbuild#2914
- use newer KoreBuild
  - `.\build.cmd -update /t:noop`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant